Skip to content

Rename routing-formulation to cuopt-routing-formulation - #1329

Merged
ramakrishnap-nv merged 2 commits into
mainfrom
skills/rename-routing-formulation
May 28, 2026
Merged

Rename routing-formulation to cuopt-routing-formulation#1329
ramakrishnap-nv merged 2 commits into
mainfrom
skills/rename-routing-formulation

Conversation

@ramakrishnap-nv

@ramakrishnap-nv ramakrishnap-nv commented May 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Rename routing-formulation to cuopt-routing-formulation

@ramakrishnap-nv @claude
skills: rename routing-formulation to cuopt-routing-formulation
40ab555 
Prefix the routing formulation skill with cuopt- to match the
product-scoped naming convention used by the other cuOpt skills
in the catalog.

Updates the directory name, the marketplace.json entry (name +
source), the AGENTS.md bullet, and the expected_skill field in
the eval. No SKILL.md content changes beyond the in-file
self-reference.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
@ramakrishnap-nv
ramakrishnap-nv requested a review from a team as a code owner May 28, 2026 23:31
@ramakrishnap-nv
ramakrishnap-nv requested a review from Iroy30 May 28, 2026 23:31
@coderabbitai

coderabbitai Bot commented May 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The PR renames the routing formulation skill from routing-formulation to cuopt-routing-formulation across marketplace configuration, agent documentation, skill metadata, and evaluation cases; it also adds a skill-card, a BENCHMARK report, and updates the signed skill bundle.

Changes

Plugin and Skill Rename to cuOpt Namespace

Layer / File(s) Summary
Plugin marketplace configuration
.claude-plugin/marketplace.json		 The plugin entry is renamed from routing-formulation to cuopt-routing-formulation and its source is updated to ./skills/cuopt-routing-formulation.
Documentation and agent listing, skill metadata, evaluation expectations
AGENTS.md, skills/cuopt-routing-formulation/SKILL.md, skills/cuopt-routing-formulation/evals/evals.json		 Agent docs, SKILL frontmatter, and eval case rt-form-eval-001-tsp-vs-vrp-vs-pdp now reference cuopt-routing-formulation.
Skill documentation and benchmark
skills/cuopt-routing-formulation/skill-card.md, skills/cuopt-routing-formulation/BENCHMARK.md		 New skill-card and BENCHMARK.md added describing the cuOpt routing formulation skill, evaluation setup, and recorded results.
Signed bundle update
skills/cuopt-routing-formulation/skill.oms.sig		 DSSE/in-toto payload and signature replaced to reflect updated subjects and resource digests for the renamed skill and added/changed documents.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • NVIDIA/cuopt#1302: Added the evaluation case rt-form-eval-001-tsp-vs-vrp-vs-pdp under the original routing-formulation skill, which is updated here to reference cuopt-routing-formulation.

Suggested labels

non-breaking, improvement

Suggested reviewers

  • Iroy30
  • bdice
  • tmckayus
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and concisely describes the main change: renaming routing-formulation to cuopt-routing-formulation, which is the primary action across all modified files.
Description check ✅ Passed The description matches the title and changeset, clearly stating the intent to rename routing-formulation to cuopt-routing-formulation to align with product naming conventions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch skills/rename-routing-formulation

Comment @coderabbitai help to get the list of available commands and usage tips.

@ramakrishnap-nv ramakrishnap-nv added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels May 28, 2026
@ramakrishnap-nv

ramakrishnap-nv commented May 28, 2026

Copy link
Copy Markdown
Collaborator Author

/nvskills-ci

@svc-nvskills-signing
Attach NVSkills validation signatures
0310cce 
Signed-off-by: nvskills-svc-account <svc-nvskills-signing@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented May 28, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ramakrishnap-nv

ramakrishnap-nv commented May 28, 2026

Copy link
Copy Markdown
Collaborator Author

/ok to test 0310cce

@ramakrishnap-nv
ramakrishnap-nv merged commit 22b1f3d into main May 28, 2026
8 of 9 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request May 28, 2026
Merged
@ramakrishnap-nv
ramakrishnap-nv deleted the skills/rename-routing-formulation branch July 9, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tmckayus tmckayus tmckayus approved these changes
@Iroy30 Iroy30 Awaiting requested review from Iroy30 Iroy30 is a code owner automatically assigned from NVIDIA/cuopt-infra-codeowners

Assignees

No one assigned

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ramakrishnap-nv @tmckayus @svc-nvskills-signing